Skip to content

feat: add output_name input to map KICS --output-name CLI option#154

Open
rasta-rocket wants to merge 1 commit intoCheckmarx:masterfrom
rasta-rocket:add_output_name
Open

feat: add output_name input to map KICS --output-name CLI option#154
rasta-rocket wants to merge 1 commit intoCheckmarx:masterfrom
rasta-rocket:add_output_name

Conversation

@rasta-rocket
Copy link
Copy Markdown

@rasta-rocket rasta-rocket commented Apr 17, 2026

Summary

Adds the missing output_name input to the GitHub Action, mapping it to the
KICS CLI --output-name flag. This allows users to customize the name of the
results output file instead of relying on the default.

Changes

  • Defined output_name input in action.yml (grouped with other output-related inputs)
  • Passed it as an arg to the Docker container in action.yml
  • Mapped INPUT_OUTPUT_NAME to --output-name in entrypoint.sh
  • Added the flag to the kics scan command
  • Updated entrypoint.sh to use the custom filename when resolving the
    copy path (CP_PATH) instead of hardcoding results.json
  • Updated processOutputPath() in src/main.js to build the JSON results
    file path using the custom name, so that post-scan features (annotations,
    PR comments, job summary) read the correct file

@rasta-rocket rasta-rocket requested a review from a team as a code owner April 17, 2026 11:52
@rasta-rocket
Copy link
Copy Markdown
Author

Not sure what is the process here
So just in case this is not forgotten

ping @cx-bruno-silva @cx-miguel-dasilva

Copy link
Copy Markdown
Collaborator

@cx-artur-ribeiro cx-artur-ribeiro left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants